Skip to content

Add Term.toExpr in Tasty #4473

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 28, 2018
Merged

Conversation

nicolasstucki
Copy link
Contributor

@nicolasstucki nicolasstucki commented May 7, 2018

Add a transformation from a Tasty.Term to an Expr[T]. A type tag quoted.Type[T] is required as type evidence.

@nicolasstucki nicolasstucki self-assigned this May 7, 2018
@nicolasstucki nicolasstucki force-pushed the tasty-to-quote branch 2 times, most recently from 08aa3c0 to e17ef67 Compare May 8, 2018 08:41
@nicolasstucki nicolasstucki force-pushed the tasty-to-quote branch 2 times, most recently from 0b57a87 to e5fbe1c Compare May 11, 2018 14:23
@nicolasstucki nicolasstucki force-pushed the tasty-to-quote branch 3 times, most recently from 545789c to 148f57e Compare May 24, 2018 09:35
@nicolasstucki nicolasstucki changed the title Add Term.toExpr and TypeTree.toType Add Term.toExpr in Tasty May 24, 2018
@nicolasstucki nicolasstucki force-pushed the tasty-to-quote branch 2 times, most recently from 80a0b3b to 99523c7 Compare May 24, 2018 14:03
@nicolasstucki nicolasstucki removed their assignment May 24, 2018
@nicolasstucki nicolasstucki requested a review from odersky May 24, 2018 14:07
@Blaisorblade
Copy link
Contributor

@nicolasstucki Can I rebase this? I managed already in master...dotty-staging:tasty-scalatest-assert, I'd force-push here.

@nicolasstucki
Copy link
Contributor Author

@Blaisorblade I rebased it. You can rebase your branch on this one.

@odersky odersky merged commit fd8f29c into scala:master May 28, 2018
@allanrenucci allanrenucci deleted the tasty-to-quote branch May 28, 2018 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants